Conversation
Goal is to simplify/give an overview before diving deeper.
…wn list indention
* Improve structure through lists * Add references to Nosto client script API * Promote Nosto helper methods for injection and parameterless attribution to reduce complexity * Move "is not recommended for SPA" higher * Add "dynamic filtering is not possible with pure GraphQL RECs" note * Add example response, making the ev1 request/response more prominent * Update comparison table
* Replace Session API drawback "Complex event tracking and attribution" with "Requires a bit more planning because page tracking and campaign injection are combined" * Add "No dynamic filtering and OCP" for GraphQL
…ly complex non-Shopify product card)
Co-authored-by: Chris Ortner <christian.ortner@nosto.com>
timowestnosto
force-pushed
the
psn-guide
branch
from
b3dd721 to
54884fe
Compare
Author
|
@copilot create a PR based on all the review feedback |
|
@timowestnosto I've opened a new pull request, #339, to work on those changes. Once the pull request is ready, I'll request review from you. |
Co-authored-by: timowestnosto <13622115+timowestnosto@users.noreply.github.com>
Address review feedback: improve clarity and add examples
|
@timowestnosto I've opened a new pull request, #340, to work on those changes. Once the pull request is ready, I'll request review from you. |
4 tasks
Co-authored-by: timowestnosto <13622115+timowestnosto@users.noreply.github.com>
[WIP] Fix feedback on Personalization guide PR
Author
|
@copilot handle the open comments |
|
@timowestnosto I've opened a new pull request, #341, to work on those changes. Once the pull request is ready, I'll request review from you. |
manikandan-ravikumar
approved these changes
Dec 2, 2025
jwittorf
requested changes
Dec 3, 2025
...plement-on-your-website/advanced-implementation/adding-support-for-customer-group-pricing.md
Outdated
Show resolved
Hide resolved
…tation/adding-support-for-customer-group-pricing.md
jwittorf
requested changes
Dec 3, 2025
jwittorf
reviewed
Dec 4, 2025
jwittorf
approved these changes
Dec 4, 2025
|
Can be merged after #330 since this includes some links to that PR. |
Author
|
@jwittorf where do you see references? I don't think this PR and 330 are compatible |
jwittorf
reviewed
Dec 4, 2025
| - [ ] Nosto account with working product sync (promotable products) | ||
| - [ ] Nosto script in the frontend (Nosto Debug Toolbar is loading) | ||
| - [ ] Knowledge of running a SPA or classic web application | ||
| - [ ] General understanding of [how Nosto works](../../getting-started/README.md) and [what components make a stable Nosto implementation](../../getting-started/building-your-implementation-plan.md#components-of-a-stable-nosto-implementation.md) |
There was a problem hiding this comment.
@timowestnosto here's the reference to #330
jwittorf
reviewed
Dec 8, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
extracted from #330